Fix freeing of records, when the given pointer is NULL.

Signed-off-by: Ewan Mellor <ewan@xensource.com>

diff --git a/tools/libxen/src/xen_host.c b/tools/libxen/src/xen_host.c
index 455f57d04ce520a7c02ceddb47c2b2fc0cf38d41..849ed1223823156ee17b65cc2abb49e46cfef8e9 100644 (file)
--- a/tools/libxen/src/xen_host.c
+++ b/tools/libxen/src/xen_host.c
@@ -76,6 +76,10 @@ const abstract_type xen_host_record_abstract_type_ =
 void
 xen_host_record_free(xen_host_record *record)
 {
+    if (record == NULL)
+    {
+        return;
+    }
     free(record->handle);
     free(record->uuid);
     free(record->name_label);

diff --git a/tools/libxen/src/xen_host_cpu.c b/tools/libxen/src/xen_host_cpu.c
index a18ded8150dab2a443d9e4bba50a7d6755944580..f2b89273453f0908ab1e67a0da4af2c5025f35c1 100644 (file)
--- a/tools/libxen/src/xen_host_cpu.c
+++ b/tools/libxen/src/xen_host_cpu.c
@@ -78,6 +78,10 @@ const abstract_type xen_host_cpu_record_abstract_type_ =
 void
 xen_host_cpu_record_free(xen_host_cpu_record *record)
 {
+    if (record == NULL)
+    {
+        return;
+    }
     free(record->handle);
     free(record->uuid);
     xen_host_record_opt_free(record->host);

diff --git a/tools/libxen/src/xen_network.c b/tools/libxen/src/xen_network.c
index 18d650ccbcf2ac1a4dcbbd0f5f791581da22db3e..c774fe72df6bbfa61ca8dd6a54f5489d72e02542 100644 (file)
--- a/tools/libxen/src/xen_network.c
+++ b/tools/libxen/src/xen_network.c
@@ -74,6 +74,10 @@ const abstract_type xen_network_record_abstract_type_ =
 void
 xen_network_record_free(xen_network_record *record)
 {
+    if (record == NULL)
+    {
+        return;
+    }
     free(record->handle);
     free(record->uuid);
     free(record->name_label);

diff --git a/tools/libxen/src/xen_pif.c b/tools/libxen/src/xen_pif.c
index 92ca6508d109d5845b9a5b4f1ea3fae636ae7b57..6210f32d659f378495b80dff6eb6f79caa5c9b81 100644 (file)
--- a/tools/libxen/src/xen_pif.c
+++ b/tools/libxen/src/xen_pif.c
@@ -80,6 +80,10 @@ const abstract_type xen_pif_record_abstract_type_ =
 void
 xen_pif_record_free(xen_pif_record *record)
 {
+    if (record == NULL)
+    {
+        return;
+    }
     free(record->handle);
     free(record->uuid);
     free(record->name);

diff --git a/tools/libxen/src/xen_sr.c b/tools/libxen/src/xen_sr.c
index d94e1238dd97f04f5252c779f8f977b291b98a1c..183ad7784372328adaca4f84955d8b18fd5d5730 100644 (file)
--- a/tools/libxen/src/xen_sr.c
+++ b/tools/libxen/src/xen_sr.c
@@ -79,6 +79,10 @@ const abstract_type xen_sr_record_abstract_type_ =
 void
 xen_sr_record_free(xen_sr_record *record)
 {
+    if (record == NULL)
+    {
+        return;
+    }
     free(record->handle);
     free(record->uuid);
     free(record->name_label);

diff --git a/tools/libxen/src/xen_user.c b/tools/libxen/src/xen_user.c
index 39a04b33f3e6e55d365a2af7f33e879c2aa7ee77..e98f0a0f86b9524358d3900c14d750a70147020b 100644 (file)
--- a/tools/libxen/src/xen_user.c
+++ b/tools/libxen/src/xen_user.c
@@ -60,6 +60,10 @@ const abstract_type xen_user_record_abstract_type_ =
 void
 xen_user_record_free(xen_user_record *record)
 {
+    if (record == NULL)
+    {
+        return;
+    }
     free(record->handle);
     free(record->uuid);
     free(record->short_name);

diff --git a/tools/libxen/src/xen_vbd.c b/tools/libxen/src/xen_vbd.c
index 228b60943d6116f23c9a0861a92828d7fac93ff3..eac4e3f26b5110a97c52c002e658397fe5adf7c9 100644 (file)
--- a/tools/libxen/src/xen_vbd.c
+++ b/tools/libxen/src/xen_vbd.c
@@ -79,6 +79,10 @@ const abstract_type xen_vbd_record_abstract_type_ =
 void
 xen_vbd_record_free(xen_vbd_record *record)
 {
+    if (record == NULL)
+    {
+        return;
+    }
     free(record->handle);
     free(record->uuid);
     xen_vm_record_opt_free(record->vm);

diff --git a/tools/libxen/src/xen_vdi.c b/tools/libxen/src/xen_vdi.c
index 11aaa48085a24eab2a70c6abc486ee713c2ad43c..9246077ec4bc3db1b2f5afcf4409c5d3315a44e9 100644 (file)
--- a/tools/libxen/src/xen_vdi.c
+++ b/tools/libxen/src/xen_vdi.c
@@ -93,6 +93,10 @@ const abstract_type xen_vdi_record_abstract_type_ =
 void
 xen_vdi_record_free(xen_vdi_record *record)
 {
+    if (record == NULL)
+    {
+        return;
+    }
     free(record->handle);
     free(record->uuid);
     free(record->name_label);

diff --git a/tools/libxen/src/xen_vif.c b/tools/libxen/src/xen_vif.c
index ac096a3cfd20f1d923240d7cc7ec0193e1313a1b..d4723d80eb039a36edf7cc9e507358c2974b6190 100644 (file)
--- a/tools/libxen/src/xen_vif.c
+++ b/tools/libxen/src/xen_vif.c
@@ -84,6 +84,10 @@ const abstract_type xen_vif_record_abstract_type_ =
 void
 xen_vif_record_free(xen_vif_record *record)
 {
+    if (record == NULL)
+    {
+        return;
+    }
     free(record->handle);
     free(record->uuid);
     free(record->name);

diff --git a/tools/libxen/src/xen_vm.c b/tools/libxen/src/xen_vm.c
index 73ccfb045403926f82844d37129c4356bbb9b537..c2f2ff7558597698f06fdfd5ce67ec52ef7c6a01 100644 (file)
--- a/tools/libxen/src/xen_vm.c
+++ b/tools/libxen/src/xen_vm.c
@@ -189,6 +189,10 @@ const abstract_type xen_vm_record_abstract_type_ =
 void
 xen_vm_record_free(xen_vm_record *record)
 {
+    if (record == NULL)
+    {
+        return;
+    }
     free(record->handle);
     free(record->uuid);
     free(record->name_label);

diff --git a/tools/libxen/src/xen_vtpm.c b/tools/libxen/src/xen_vtpm.c
index 75e90026f30d07a3a522dbfc91b3a47c2b46f179..b3baabacd5c2333f1352f3b6c89bb4efacd31819 100644 (file)
--- a/tools/libxen/src/xen_vtpm.c
+++ b/tools/libxen/src/xen_vtpm.c
@@ -69,6 +69,10 @@ const abstract_type xen_vtpm_record_abstract_type_ =
 void
 xen_vtpm_record_free(xen_vtpm_record *record)
 {
+    if (record == NULL)
+    {
+        return;
+    }
     free(record->handle);
     free(record->uuid);
     xen_vm_record_opt_free(record->vm);